Skip to content

Fix editor-backed tool E2E flows#100

Merged
vuon9 merged 4 commits into
mainfrom
cdx/fix-editor-e2e-contract
May 21, 2026
Merged

Fix editor-backed tool E2E flows#100
vuon9 merged 4 commits into
mainfrom
cdx/fix-editor-e2e-contract

Conversation

@vuon9
Copy link
Copy Markdown
Owner

@vuon9 vuon9 commented May 21, 2026

Summary

  • add stable test/accessibility hooks to CodeMirror-backed editors and affected tool panes
  • fix read-only highlighted output panes so they mount while empty and update when output arrives
  • update end-user E2E specs to use shared editor helpers instead of raw textarea selectors
  • add accessible Barcode layout toggle labeling and disable empty Data Generator copy icon

Verification

  • cd frontend && bun run format
  • cd frontend && node_modules/@playwright/test/cli.js test e2e/codeEncoder.spec.js e2e/codeConverter.spec.js e2e/hashGenerator.spec.js e2e/codeEncrypter.spec.js e2e/textUtilities.spec.js e2e/textDiffChecker.spec.js e2e/jwtDebugger.spec.js e2e/codeFormatter.spec.js e2e/dataGenerator.spec.js e2e/barcodeGenerator.spec.js e2e/regExpTester.spec.js (138 passed)
  • cd frontend && node_modules/@playwright/test/cli.js test (217 passed)
  • cd frontend && node_modules/vitest/vitest.mjs run (31 passed)
  • cd frontend && node_modules/vite/bin/vite.js build --mode production
  • go test ./internal/... ./service/... ./pkg/...

Known unrelated issue

  • go test ./... still fails in cmd/testserver because cmd/testserver/main.go references service.NewConversionService, which is not part of this change.

vuon9 added 2 commits May 21, 2026 15:43
Add stable CodeMirror test attributes and shared Playwright editor helpers. Fix highlighted read-only panes so empty outputs mount and update when generated content arrives. Refresh affected end-user feature specs around encoder, converter, hash, encryption, JWT, formatter, data generation, and text tools.
CI now builds against the current main branch, where Dependabot bumped Wails to v3.0.0-alpha.8.3. The app still depends on APIs such as RegisterEvent, RegisterService, and app.Event that are present in v3.0.0-alpha.88, so restore that version to keep the build green.
@vuon9 vuon9 force-pushed the cdx/fix-editor-e2e-contract branch from 1e2c0b1 to 18394f1 Compare May 21, 2026 08:47
@github-actions
Copy link
Copy Markdown
Contributor

Code Coverage

Package Line Rate Health
devtoolbox/internal/barcode 93%
devtoolbox/internal/codeformatter 80%
devtoolbox/internal/converter 65%
devtoolbox/internal/datagenerator 81%
devtoolbox/internal/datetimeconverter 75%
devtoolbox/internal/jwt 47%
devtoolbox/internal/numberconverter 0%
devtoolbox/internal/settings 0%
devtoolbox/internal/themes 0%
Summary 68% (3613 / 5294)

Minimum allowed line rate is 60%

@vuon9 vuon9 merged commit 4ba76c5 into main May 21, 2026
6 checks passed
@vuon9 vuon9 deleted the cdx/fix-editor-e2e-contract branch May 21, 2026 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant